Pods 3.2.8 #7347

PodsBot · 2024-08-31T01:26:11Z

Automated changes by create-pull-request GitHub action

what-the-diff · 2024-08-31T01:26:27Z

PR Summary

Updated Coding Standards
The former coding standards used in the project, TribalScents, were replaced with the StellarWP standard. As part of this change, some rules were also excluded.
Changed dependencies
The project no longer depends on "the-events-calendar/coding-standards". Instead, it now utilizes "stellarwp/coding-standards".
Improved Functionality in 'general.php'
The pods_shortcode_run function has been updated to increase its reliability when assigning variables and checking conditions.
Version Upgrade
The plugin, as well as the version in the package.json file and the stable tag version in the readme.txt, were all upgraded from 3.2.7 to 3.2.8-a-1.
Added Content Formatting Option in 'Base.php'
A method, should_autop, was implemented to decide whether the wpautop (a function that automatically adds line breaks and paragraph tags to text) should be applied to block content output.
Enhanced Content Rendering in 'Field.php', 'Item_List.php', and 'Related_Item_List.php'
The render method in these files now calls the previously mentioned should_autop method when determining how to format the content.

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
Public method name "Base::should_autop" is not in camel caps format

pods/src/Pods/Blocks/Types/Base.php

Line 284 in 8b99c43

public function should_autop( $content, array $attributes = [] ): bool {

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionDeclaration.BraceOnSameLine
Opening brace should be on a new line

pods/src/Pods/Blocks/Types/Base.php

Line 284 in 8b99c43

public function should_autop( $content, array $attributes = [] ): bool {

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 285 in 8b99c43

$should_autop = (

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 286 in 8b99c43

is_string( $content )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 287 in 8b99c43

&& false === strpos( $content, '<div' )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 288 in 8b99c43

&& false === strpos( $content, '<ul' )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 289 in 8b99c43

&& false === strpos( $content, '<ol' )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 290 in 8b99c43

&& false === strpos( $content, '<h' )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 291 in 8b99c43

&& false === strpos( $content, '<p' )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 292 in 8b99c43

);

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 294 in 8b99c43

/**

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 295 in 8b99c43

* Allow filtering whether to apply wpautop to the block content output.

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 296 in 8b99c43

*

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 297 in 8b99c43

    
           		 * This is used for things like Field block render or List block render of the no items found message.

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.TooLong
Line exceeds 85 characters; contains 110 characters

pods/src/Pods/Blocks/Types/Base.php

Line 297 in 8b99c43

    
           		 * This is used for things like Field block render or List block render of the no items found message.

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 298 in 8b99c43

*

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 299 in 8b99c43

* @since TBD

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 300 in 8b99c43

*

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 301 in 8b99c43

    
           		 * @param bool         $should_autop Whether to apply wpautop to the block content output.

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.TooLong
Line exceeds 85 characters; contains 98 characters

pods/src/Pods/Blocks/Types/Base.php

Line 301 in 8b99c43

    
           		 * @param bool         $should_autop Whether to apply wpautop to the block content output.

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 302 in 8b99c43

* @param string|mixed $content The content to determine whether to autop.

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.TooLong
Line exceeds 85 characters; contains 87 characters

pods/src/Pods/Blocks/Types/Base.php

Line 302 in 8b99c43

* @param string|mixed $content The content to determine whether to autop.

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 303 in 8b99c43

    
           		 * @param array        $attributes   The Pods render attributes that will be used.

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.TooLong
Line exceeds 85 characters; contains 90 characters

pods/src/Pods/Blocks/Types/Base.php

Line 303 in 8b99c43

    
           		 * @param array        $attributes   The Pods render attributes that will be used.

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 304 in 8b99c43

*/

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 305 in 8b99c43

    
           return (bool) apply_filters( 'pods_blocks_should_autop', $should_autop, $content, $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.TooLong
Line exceeds 85 characters; contains 104 characters

pods/src/Pods/Blocks/Types/Base.php

Line 305 in 8b99c43

    
           return (bool) apply_filters( 'pods_blocks_should_autop', $should_autop, $content, $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 306 in 8b99c43

}

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Field.php

Line 215 in 8b99c43

if ( $this->should_autop( $content, $attributes ) ) {

[phpcs] _{reported by reviewdog 🐶}
PEAR.ControlStructures.MultiLineCondition.SpacingAfterOpenBrace
First condition of a multi-line IF statement must directly follow the opening parenthesis

pods/src/Pods/Blocks/Types/Field.php

Line 215 in 8b99c43

if ( $this->should_autop( $content, $attributes ) ) {

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Item_List.php

Line 429 in 8b99c43

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
PEAR.ControlStructures.MultiLineCondition.SpacingAfterOpenBrace
First condition of a multi-line IF statement must directly follow the opening parenthesis

pods/src/Pods/Blocks/Types/Item_List.php

Line 429 in 8b99c43

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.TooLong
Line exceeds 85 characters; contains 103 characters

pods/src/Pods/Blocks/Types/Item_List.php

Line 429 in 8b99c43

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Item_List.php

Line 430 in 8b99c43

$attributes['not_found'] = wpautop( $attributes['not_found'], $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.TooLong
Line exceeds 85 characters; contains 88 characters

pods/src/Pods/Blocks/Types/Item_List.php

Line 430 in 8b99c43

$attributes['not_found'] = wpautop( $attributes['not_found'], $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Item_List.php

Line 431 in 8b99c43

}

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 347 in 8b99c43

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
PEAR.ControlStructures.MultiLineCondition.SpacingAfterOpenBrace
First condition of a multi-line IF statement must directly follow the opening parenthesis

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 347 in 8b99c43

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.TooLong
Line exceeds 85 characters; contains 103 characters

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 347 in 8b99c43

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 348 in 8b99c43

$attributes['not_found'] = wpautop( $attributes['not_found'], $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.TooLong
Line exceeds 85 characters; contains 88 characters

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 348 in 8b99c43

$attributes['not_found'] = wpautop( $attributes['not_found'], $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 349 in 8b99c43

}

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 285 in 6dee8cc

$should_autop = (

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 286 in 6dee8cc

is_string( $content )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 287 in 6dee8cc

&& false === strpos( $content, '<div' )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 288 in 6dee8cc

&& false === strpos( $content, '<ul' )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 289 in 6dee8cc

&& false === strpos( $content, '<ol' )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 290 in 6dee8cc

&& false === strpos( $content, '<h' )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 291 in 6dee8cc

&& false === strpos( $content, '<p' )

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 292 in 6dee8cc

);

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 294 in 6dee8cc

/**

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 295 in 6dee8cc

* Allow filtering whether to apply wpautop to the block content output.

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 296 in 6dee8cc

*

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 297 in 6dee8cc

    
           		 * This is used for things like Field block render or List block render of the no items found message.

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.MaxExceeded
Line exceeds maximum limit of 100 characters; contains 110 characters

pods/src/Pods/Blocks/Types/Base.php

Line 297 in 6dee8cc

    
           		 * This is used for things like Field block render or List block render of the no items found message.

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 298 in 6dee8cc

*

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 299 in 6dee8cc

* @since TBD

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 300 in 6dee8cc

*

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 301 in 6dee8cc

    
           		 * @param bool         $should_autop Whether to apply wpautop to the block content output.

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 302 in 6dee8cc

* @param string|mixed $content The content to determine whether to autop.

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 303 in 6dee8cc

    
           		 * @param array        $attributes   The Pods render attributes that will be used.

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 304 in 6dee8cc

*/

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 305 in 6dee8cc

    
           return (bool) apply_filters( 'pods_blocks_should_autop', $should_autop, $content, $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.Formatting.NoSpaceAfterCast.SpaceFound
A cast statement must not be followed by a space

pods/src/Pods/Blocks/Types/Base.php

Line 305 in 6dee8cc

    
           return (bool) apply_filters( 'pods_blocks_should_autop', $should_autop, $content, $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.MaxExceeded
Line exceeds maximum limit of 100 characters; contains 104 characters

pods/src/Pods/Blocks/Types/Base.php

Line 305 in 6dee8cc

    
           return (bool) apply_filters( 'pods_blocks_should_autop', $should_autop, $content, $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Base.php

Line 306 in 6dee8cc

}

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Field.php

Line 215 in 6dee8cc

if ( $this->should_autop( $content, $attributes ) ) {

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Item_List.php

Line 429 in 6dee8cc

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.MaxExceeded
Line exceeds maximum limit of 100 characters; contains 103 characters

pods/src/Pods/Blocks/Types/Item_List.php

Line 429 in 6dee8cc

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Item_List.php

Line 430 in 6dee8cc

$attributes['not_found'] = wpautop( $attributes['not_found'], $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Item_List.php

Line 431 in 6dee8cc

}

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 347 in 6dee8cc

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
Generic.Files.LineLength.MaxExceeded
Line exceeds maximum limit of 100 characters; contains 103 characters

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 347 in 6dee8cc

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 348 in 6dee8cc

$attributes['not_found'] = wpautop( $attributes['not_found'], $attributes );

[phpcs] _{reported by reviewdog 🐶}
Generic.WhiteSpace.DisallowTabIndent.TabsUsed
Spaces must be used to indent lines; tabs are not allowed

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 349 in 6dee8cc

}

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.SpaceAfterOpenBracket
Space after opening parenthesis of function call prohibited

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 347 in 06183dd

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.SpaceBeforeCloseBracket
Expected 0 spaces before closing parenthesis; 1 found

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 347 in 06183dd

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.SpaceAfterOpenBracket
Space after opening parenthesis of function call prohibited

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 347 in 06183dd

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.SpaceBeforeCloseBracket
Expected 0 spaces before closing parenthesis; 1 found

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 347 in 06183dd

    
           if ( ! empty( $attributes['not_found'] ) && $this->should_autop( $attributes['not_found'] ) ) {

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.SpaceAfterOpenBracket
Space after opening parenthesis of function call prohibited

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 348 in 06183dd

$attributes['not_found'] = wpautop( $attributes['not_found'], $attributes );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.SpaceBeforeCloseBracket
Expected 0 spaces before closing parenthesis; 1 found

pods/src/Pods/Blocks/Types/Related_Item_List.php

Line 348 in 06183dd

$attributes['not_found'] = wpautop( $attributes['not_found'], $attributes );

PodsBot · 2024-09-10T04:16:51Z

ui/admin/setup-add.php

+											<?php echo( $submit_from_linked ? 'class="hidden"' : '' ); ?>>
+											<h2 class="pods-wizard-one-click-actions-heading"><?php esc_html_e( 'One-Click Extend', 'pods' ); ?></h2>
+											<ul class="pods-wizard-one-click-actions">
+												<?php foreach ( $quick_actions as $quick_action_key => $quick_action ) : ?>


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$quick_action_key".

PodsBot · 2024-09-10T04:16:51Z

ui/admin/setup-add.php

+											<?php echo( $submit_from_linked ? 'class="hidden"' : '' ); ?>>
+											<h2 class="pods-wizard-one-click-actions-heading"><?php esc_html_e( 'One-Click Extend', 'pods' ); ?></h2>
+											<ul class="pods-wizard-one-click-actions">
+												<?php foreach ( $quick_actions as $quick_action_key => $quick_action ) : ?>


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$quick_action".

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'pods_v'.

pods/components/Roles/ui/add.php

Line 41 in 6ccfcd8

echo PodsForm::field( 'role_label', pods_v( 'role_label', 'post' ), 'text', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Roles/ui/add.php

Line 41 in 6ccfcd8

echo PodsForm::field( 'role_label', pods_v( 'role_label', 'post' ), 'text', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Roles/ui/add.php

Line 41 in 6ccfcd8

echo PodsForm::field( 'role_label', pods_v( 'role_label', 'post' ), 'text', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Roles/ui/add.php

Line 41 in 6ccfcd8

echo PodsForm::field( 'role_label', pods_v( 'role_label', 'post' ), 'text', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Roles/ui/add.php

Line 50 in 6ccfcd8

echo PodsForm::field( 'role_name', pods_v( 'role_name', 'post' ), 'slug', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Roles/ui/add.php

Line 50 in 6ccfcd8

echo PodsForm::field( 'role_name', pods_v( 'role_name', 'post' ), 'slug', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'pods_v'.

pods/components/Roles/ui/add.php

Line 50 in 6ccfcd8

echo PodsForm::field( 'role_name', pods_v( 'role_name', 'post' ), 'slug', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Roles/ui/add.php

Line 50 in 6ccfcd8

echo PodsForm::field( 'role_name', pods_v( 'role_name', 'post' ), 'slug', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Roles/ui/add.php

Line 50 in 6ccfcd8

echo PodsForm::field( 'role_name', pods_v( 'role_name', 'post' ), 'slug', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Roles/ui/add.php

Line 50 in 6ccfcd8

echo PodsForm::field( 'role_name', pods_v( 'role_name', 'post' ), 'slug', [

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.EmbeddedPhp.ContentAfterOpen
Opening PHP tag must be on a line by itself

pods/components/Roles/ui/add.php

Line 91 in 6ccfcd8

    
           <?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Roles/ui/add.php

Line 91 in 6ccfcd8

    
           <?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Roles/ui/add.php

Line 91 in 6ccfcd8

    
           <?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$capability'.

pods/components/Roles/ui/add.php

Line 91 in 6ccfcd8

    
           <?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'pods_v'.

pods/components/Roles/ui/add.php

Line 91 in 6ccfcd8

    
           <?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Roles/ui/add.php

Line 91 in 6ccfcd8

    
           <?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Roles/ui/add.php

Line 91 in 6ccfcd8

    
           <?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Roles/ui/add.php

Line 91 in 6ccfcd8

    
           <?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$obj'

pods/components/Roles/ui/edit.php

Line 24 in 6ccfcd8

echo $obj->message( wp_kses_post( $message ) );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$obj'.

pods/components/Roles/ui/edit.php

Line 24 in 6ccfcd8

echo $obj->message( wp_kses_post( $message ) );

[phpcs] _{reported by reviewdog 🐶}
WordPressVIPMinimum.Security.ProperEscapingFunction.htmlAttrNotByEscHTML
Wrong escaping function. HTML attributes should be escaped by esc_attr(), not by esc_html_e().

pods/components/Roles/ui/edit.php

Line 46 in 6ccfcd8

    
           <input type="submit" name="publish" id="publish" class="button-primary" value="<?php esc_html_e( 'Save', 'pods' ); ?>" accesskey="p" />

[phpcs] _{reported by reviewdog 🐶}
Squiz.Strings.EchoedStrings.HasBracket
Echoed strings should not be bracketed

pods/ui/admin/setup-add.php

Line 147 in 6ccfcd8

    
           <div class="pods-wizard-option<?php echo ( ! empty( $quick_actions ) ? ' pods-wizard-option-with-secondary' : '' ); ?>">

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '!'

pods/ui/admin/setup-add.php

Line 147 in 6ccfcd8

    
           <div class="pods-wizard-option<?php echo ( ! empty( $quick_actions ) ? ' pods-wizard-option-with-secondary' : '' ); ?>">

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/ui/admin/setup-add.php

Line 147 in 6ccfcd8

    
           <div class="pods-wizard-option<?php echo ( ! empty( $quick_actions ) ? ' pods-wizard-option-with-secondary' : '' ); ?>">

[phpcs] _{reported by reviewdog 🐶}
Squiz.Strings.EchoedStrings.HasBracket
Echoed strings should not be bracketed

pods/ui/admin/setup-add.php

Line 158 in 6ccfcd8

<?php echo ( $submit_from_linked ? 'class="hidden"' : '' ); ?>>

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$submit_from_linked'

pods/ui/admin/setup-add.php

Line 158 in 6ccfcd8

<?php echo ( $submit_from_linked ? 'class="hidden"' : '' ); ?>>

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/ui/admin/setup-add.php

Line 158 in 6ccfcd8

<?php echo ( $submit_from_linked ? 'class="hidden"' : '' ); ?>>

PodsBot · 2024-09-12T20:35:47Z

components/Migrate-CPTUI/ui/wizard.php

 											<ul>
 												<?php
 												foreach ( $post_types as $post_type ) {
-													$post_type_name  = pods_var_raw( 'name', $post_type );
-													$post_type_label = pods_var_raw( 'label', $post_type, ucwords( str_replace( '_', ' ', $post_type_name ) ) );
+													$post_type_name  = pods_v( 'name', $post_type );


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$post_type_name".

PodsBot · 2024-09-12T20:35:47Z

components/Migrate-CPTUI/ui/wizard.php

-													$post_type_name  = pods_var_raw( 'name', $post_type );
-													$post_type_label = pods_var_raw( 'label', $post_type, ucwords( str_replace( '_', ' ', $post_type_name ) ) );
+													$post_type_name  = pods_v( 'name', $post_type );
+													$post_type_label = pods_v( 'label', $post_type, ucwords( str_replace( '_', ' ', $post_type_name ) ) );


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$post_type_label".

PodsBot · 2024-09-12T20:35:47Z

components/Migrate-CPTUI/ui/wizard.php

 													?>
 													<li>
 														<div class="pods-field pods-boolean">
 															<?php
-															echo PodsForm::field( 'post_type[' . $post_type_name . ']', pods_var_raw( 'post_type[' . $post_type_name . ']', 'post', true ), 'boolean', [
+															echo PodsForm::field( 'post_type[' . $post_type_name . ']', pods_v( 'post_type[' . $post_type_name . ']', 'post', true ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

PodsBot · 2024-09-12T20:35:48Z

components/Migrate-CPTUI/ui/wizard.php

 													?>
 													<li>
 														<div class="pods-field pods-boolean">
 															<?php
-															echo PodsForm::field( 'post_type[' . $post_type_name . ']', pods_var_raw( 'post_type[' . $post_type_name . ']', 'post', true ), 'boolean', [
+															echo PodsForm::field( 'post_type[' . $post_type_name . ']', pods_v( 'post_type[' . $post_type_name . ']', 'post', true ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

PodsBot · 2024-09-12T20:35:48Z

components/Migrate-CPTUI/ui/wizard.php

 													?>
 													<li>
 														<div class="pods-field pods-boolean">
 															<?php
-															echo PodsForm::field( 'post_type[' . $post_type_name . ']', pods_var_raw( 'post_type[' . $post_type_name . ']', 'post', true ), 'boolean', [
+															echo PodsForm::field( 'post_type[' . $post_type_name . ']', pods_v( 'post_type[' . $post_type_name . ']', 'post', true ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$post_type_name'.

PodsBot · 2024-09-12T20:35:48Z

components/Migrate-CPTUI/ui/wizard.php

 													?>
 													<li>
 														<?php
-														echo PodsForm::field( 'taxonomy[' . $taxonomy_name . ']', pods_var_raw( 'taxonomy[' . $taxonomy_name . ']', 'post', true ), 'boolean', [
+														echo PodsForm::field( 'taxonomy[' . $taxonomy_name . ']', pods_v( 'taxonomy[' . $taxonomy_name . ']', 'post', true ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-09-12T20:35:49Z

components/Roles/Roles.php

@@ -321,9 +325,9 @@

 		$capabilities = $this->get_capabilities();

-		$params->capabilities = (array) pods_var_raw( 'capabilities', $params, array() );
+		$params->capabilities = (array) pods_v( 'capabilities', $params, array() );


[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

PodsBot · 2024-09-12T20:35:49Z

components/Roles/Roles.php


-		$params->custom_capabilities = (array) pods_var_raw( 'custom_capabilities', $params, array() );
+		$params->custom_capabilities = (array) pods_v( 'custom_capabilities', $params, array() );


[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

PodsBot · 2024-09-12T20:35:49Z

components/Roles/ui/add.php

-									echo PodsForm::field( 'role_label', pods_var_raw( 'role_label', 'post' ), 'text', [
-										'class'       => 'pods-validate pods-validate-required',
-										'disable_dfv' => true,
+									echo PodsForm::field( 'role_label', pods_v( 'role_label', 'post' ), 'text', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

PodsBot · 2024-09-12T20:35:49Z

components/Roles/ui/add.php

-									echo PodsForm::field( 'role_label', pods_var_raw( 'role_label', 'post' ), 'text', [
-										'class'       => 'pods-validate pods-validate-required',
-										'disable_dfv' => true,
+									echo PodsForm::field( 'role_label', pods_v( 'role_label', 'post' ), 'text', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

PodsBot · 2024-09-12T20:37:52Z

components/Roles/ui/add.php

-									echo PodsForm::field( 'role_label', pods_var_raw( 'role_label', 'post' ), 'text', [
-										'class'       => 'pods-validate pods-validate-required',
-										'disable_dfv' => true,
+									echo PodsForm::field( 'role_label', pods_v( 'role_label', 'post' ), 'text', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'pods_v'.

PodsBot · 2024-09-12T20:37:52Z

components/Roles/ui/add.php

-									echo PodsForm::field( 'role_label', pods_var_raw( 'role_label', 'post' ), 'text', [
-										'class'       => 'pods-validate pods-validate-required',
-										'disable_dfv' => true,
+									echo PodsForm::field( 'role_label', pods_v( 'role_label', 'post' ), 'text', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-09-12T20:37:52Z

components/Roles/ui/add.php

-									echo PodsForm::field( 'role_label', pods_var_raw( 'role_label', 'post' ), 'text', [
-										'class'       => 'pods-validate pods-validate-required',
-										'disable_dfv' => true,
+									echo PodsForm::field( 'role_label', pods_v( 'role_label', 'post' ), 'text', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-09-12T20:37:52Z

components/Roles/ui/add.php

-									echo PodsForm::field( 'role_label', pods_var_raw( 'role_label', 'post' ), 'text', [
-										'class'       => 'pods-validate pods-validate-required',
-										'disable_dfv' => true,
+									echo PodsForm::field( 'role_label', pods_v( 'role_label', 'post' ), 'text', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-09-12T20:37:52Z

components/Roles/ui/add.php

-										'attributes'  => [ 'data-sluggable' => 'role_label' ],
-										'class'       => 'pods-validate pods-validate-required pods-slugged-lower',
-										'disable_dfv' => true,
+									echo PodsForm::field( 'role_name', pods_v( 'role_name', 'post' ), 'slug', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

PodsBot · 2024-09-12T20:37:53Z

components/Roles/ui/add.php

@@ -90,7 +88,7 @@
 												$zebra = ( ! $zebra );
 												?>
 												<li class="pods-zebra-<?php echo esc_attr( $class ); ?>" data-capability="<?php echo esc_attr( $capability ); ?>">
-													<?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_var_raw( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [
+													<?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-09-12T20:37:53Z

components/Roles/ui/add.php

@@ -90,7 +88,7 @@
 												$zebra = ( ! $zebra );
 												?>
 												<li class="pods-zebra-<?php echo esc_attr( $class ); ?>" data-capability="<?php echo esc_attr( $capability ); ?>">
-													<?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_var_raw( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [
+													<?php echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-09-12T20:37:53Z

components/Roles/ui/edit.php

@@ -21,11 +21,11 @@

 				$message = sprintf( __( '<strong>Success!</strong> %1$s %2$s successfully.', 'pods' ), $obj->item, $action );

-				echo $obj->message( $message );
+				echo $obj->message( wp_kses_post( $message ) );


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$obj'

PodsBot · 2024-09-12T20:37:53Z

components/Roles/ui/edit.php

@@ -21,11 +21,11 @@

 				$message = sprintf( __( '<strong>Success!</strong> %1$s %2$s successfully.', 'pods' ), $obj->item, $action );

-				echo $obj->message( $message );
+				echo $obj->message( wp_kses_post( $message ) );


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$obj'.

PodsBot · 2024-09-12T20:37:53Z

components/Roles/ui/edit.php


 								<div class="inside">
 									<div class="submitbox" id="submitpost">
 										<div id="minor-publishing">
 											<div id="major-publishing-actions">
 												<div id="publishing-action">
 													<img class="waiting" src="<?php echo esc_url( admin_url( 'images/wpspin_light.gif' ) ); ?>" alt="" />
-													<input type="submit" name="publish" id="publish" class="button-primary" value="<?php _e( 'Save', 'pods' ); ?>" accesskey="p" />
+													<input type="submit" name="publish" id="publish" class="button-primary" value="<?php esc_html_e( 'Save', 'pods' ); ?>" accesskey="p" />


[phpcs] _{reported by reviewdog 🐶}
WordPressVIPMinimum.Security.ProperEscapingFunction.htmlAttrNotByEscHTML
Wrong escaping function. HTML attributes should be escaped by esc_attr(), not by esc_html_e().

ui/js/jquery.pods.js

PodsBot · 2024-09-13T20:26:43Z

components/Roles/ui/edit.php

@@ -99,7 +99,7 @@
 														?>
 														<li class="pods-zebra-<?php echo esc_attr( $class ); ?>" data-capability="<?php echo esc_attr( $capability ); ?>">
 															<?php
-															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_var_raw( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [
+															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

PodsBot · 2024-09-13T20:26:43Z

components/Roles/ui/edit.php

@@ -99,7 +99,7 @@
 														?>
 														<li class="pods-zebra-<?php echo esc_attr( $class ); ?>" data-capability="<?php echo esc_attr( $capability ); ?>">
 															<?php
-															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_var_raw( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [
+															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

PodsBot · 2024-09-13T20:26:43Z

components/Roles/ui/edit.php

@@ -99,7 +99,7 @@
 														?>
 														<li class="pods-zebra-<?php echo esc_attr( $class ); ?>" data-capability="<?php echo esc_attr( $capability ); ?>">
 															<?php
-															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_var_raw( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [
+															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$capability'.

PodsBot · 2024-09-13T20:26:43Z

components/Roles/ui/edit.php

@@ -99,7 +99,7 @@
 														?>
 														<li class="pods-zebra-<?php echo esc_attr( $class ); ?>" data-capability="<?php echo esc_attr( $capability ); ?>">
 															<?php
-															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_var_raw( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [
+															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'pods_v'.

PodsBot · 2024-09-13T20:26:44Z

components/Roles/ui/edit.php

@@ -99,7 +99,7 @@
 														?>
 														<li class="pods-zebra-<?php echo esc_attr( $class ); ?>" data-capability="<?php echo esc_attr( $capability ); ?>">
 															<?php
-															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_var_raw( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [
+															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-09-13T20:26:44Z

components/Roles/ui/edit.php

@@ -99,7 +99,7 @@
 														?>
 														<li class="pods-zebra-<?php echo esc_attr( $class ); ?>" data-capability="<?php echo esc_attr( $capability ); ?>">
 															<?php
-															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_var_raw( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [
+															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-09-13T20:26:44Z

components/Roles/ui/edit.php

@@ -99,7 +99,7 @@
 														?>
 														<li class="pods-zebra-<?php echo esc_attr( $class ); ?>" data-capability="<?php echo esc_attr( $capability ); ?>">
 															<?php
-															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_var_raw( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [
+															echo PodsForm::field( 'capabilities[' . $capability . ']', pods_v( 'capabilities[' . $capability . ']', 'post', $checked ), 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-09-14T01:55:57Z

ui/admin/setup-add.php

 									</a>
 								</div>

-								<div class="pods-wizard-option">
+								<div class="pods-wizard-option<?php echo ! empty( $quick_actions ) ? ' pods-wizard-option-with-secondary' : ''; ?>">


[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

PodsBot · 2024-09-14T01:55:57Z

ui/admin/setup-add.php

+
+									<?php if ( ! empty( $quick_actions ) ) : ?>
+										<div id="pods-wizard-quick-actions"
+											<?php echo $submit_from_linked ? 'class="hidden"' : ''; ?>>


[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

PodsBot · 2024-09-14T01:55:57Z

ui/admin/setup-add.php

@@ -569,6 +567,14 @@
 								</div>
 							</div>
 						</div>
+
+						<div id="pods-wizard-quick-actions-saving-in-progress"
+							<?php echo ! $submit_from_linked ? 'class="hidden"' : ''; ?>>


[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

PodsBot · 2024-09-27T05:06:16Z

components/Templates/includes/element-view_template.php

@@ -5,8 +5,12 @@
 * @package Pods_templates
 */

+$has_php = false;


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$has_php".

PodsBot · 2024-09-27T05:06:17Z

components/Templates/includes/element-view_template.php

 if ( isset( $content ) ) {
+	$has_php = false !== strpos( $content, '<?' );


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$has_php".

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Migrate-PHP/ui/wizard.php

Line 9 in 17972ea

    
           <?php echo PodsForm::field( '_wpnonce', wp_create_nonce( 'pods-component-' . $component . '-' . $method ), 'hidden' ); ?>

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'wp_create_nonce'.

pods/components/Migrate-PHP/ui/wizard.php

Line 9 in 17972ea

    
           <?php echo PodsForm::field( '_wpnonce', wp_create_nonce( 'pods-component-' . $component . '-' . $method ), 'hidden' ); ?>

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 9 in 17972ea

    
           <?php echo PodsForm::field( '_wpnonce', wp_create_nonce( 'pods-component-' . $component . '-' . $method ), 'hidden' ); ?>

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data".

pods/components/Migrate-PHP/ui/wizard.php

Line 61 in 17972ea

$data = $pod_templates;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data_name".

pods/components/Migrate-PHP/ui/wizard.php

Line 62 in 17972ea

$data_name = 'templates';

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 80 in 17972ea

$zebra = false;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$item".

pods/components/Migrate-PHP/ui/wizard.php

Line 82 in 17972ea

foreach ( $data as $item ) {

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$checked".

pods/components/Migrate-PHP/ui/wizard.php

Line 83 in 17972ea

$checked = true;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$class".

pods/components/Migrate-PHP/ui/wizard.php

Line 85 in 17972ea

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 85 in 17972ea

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 87 in 17972ea

$zebra = ( ! $zebra );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Migrate-PHP/ui/wizard.php

Line 91 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/ui/wizard.php

Line 91 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 91 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 91 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 91 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 91 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 91 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 92 in 17972ea

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 92 in 17972ea

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 92 in 17972ea

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 94 in 17972ea

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 94 in 17972ea

] );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data".

pods/components/Migrate-PHP/ui/wizard.php

Line 109 in 17972ea

$data = $pod_pages;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data_name".

pods/components/Migrate-PHP/ui/wizard.php

Line 110 in 17972ea

$data_name = 'pages';

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 128 in 17972ea

$zebra = false;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$item".

pods/components/Migrate-PHP/ui/wizard.php

Line 130 in 17972ea

foreach ( $data as $item ) {

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$checked".

pods/components/Migrate-PHP/ui/wizard.php

Line 131 in 17972ea

$checked = true;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$class".

pods/components/Migrate-PHP/ui/wizard.php

Line 133 in 17972ea

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 133 in 17972ea

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 135 in 17972ea

$zebra = ( ! $zebra );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Migrate-PHP/ui/wizard.php

Line 139 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/ui/wizard.php

Line 139 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 139 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 139 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 139 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 139 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 139 in 17972ea

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 140 in 17972ea

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 140 in 17972ea

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 140 in 17972ea

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 142 in 17972ea

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 142 in 17972ea

] );

[phpcs] _{reported by reviewdog 🐶}
Universal.NamingConventions.NoReservedKeywordParameterNames.objectFound
It is recommended not to use reserved keyword "object" as function parameter name. Found: $object

pods/components/Pages.php

Line 1058 in 17972ea

public static function object_to_page( Page $object ): array {

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.InlineComment.InvalidEndChar
Inline comments must end in full-stops, exclamation marks, or question marks

pods/components/Pages.php

Line 1066 in 17972ea

// phpcode is deprecated

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1071 in 17972ea

    
           'admin_only'              => (boolean) get_post_meta( $id, 'admin_only', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1072 in 17972ea

    
           'restrict_role'           => (boolean) get_post_meta( $id, 'restrict_role', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1073 in 17972ea

    
           'restrict_capability'     => (boolean) get_post_meta( $id, 'restrict_capability', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1076 in 17972ea

    
           'restrict_redirect'       => (boolean) get_post_meta( $id, 'restrict_redirect', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1077 in 17972ea

    
           'restrict_redirect_login' => (boolean) get_post_meta( $id, 'restrict_redirect_login', true ),

[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

pods/components/Templates/Templates.php

Line 714 in 17972ea

$default_templates = array(

PodsBot · 2024-10-01T03:55:43Z

components/Migrate-PHP/ui/wizard.php

+
+	<form action="" method="post" class="pods-submittable">
+		<div class="pods-submittable-fields">
+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FileComment.Missing
Missing file doc comment

PodsBot · 2024-10-01T03:55:43Z

components/Migrate-PHP/ui/wizard.php

+
+	<form action="" method="post" class="pods-submittable">
+		<div class="pods-submittable-fields">
+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

PodsBot · 2024-10-01T03:55:43Z

components/Migrate-PHP/ui/wizard.php

+
+	<form action="" method="post" class="pods-submittable">
+		<div class="pods-submittable-fields">
+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

PodsBot · 2024-10-01T03:55:43Z

components/Migrate-PHP/ui/wizard.php

+	<form action="" method="post" class="pods-submittable">
+		<div class="pods-submittable-fields">
+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>
+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

PodsBot · 2024-10-01T03:55:43Z

components/Migrate-PHP/ui/wizard.php

+	<form action="" method="post" class="pods-submittable">
+		<div class="pods-submittable-fields">
+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>
+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$component'.

PodsBot · 2024-10-01T03:55:43Z

components/Migrate-PHP/ui/wizard.php

+	<form action="" method="post" class="pods-submittable">
+		<div class="pods-submittable-fields">
+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>
+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

PodsBot · 2024-10-01T03:55:43Z

components/Migrate-PHP/ui/wizard.php

+		<div class="pods-submittable-fields">
+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>
+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>
+			<?php echo PodsForm::field( 'method', $method, 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

PodsBot · 2024-10-01T03:55:43Z

components/Migrate-PHP/ui/wizard.php

+		<div class="pods-submittable-fields">
+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>
+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>
+			<?php echo PodsForm::field( 'method', $method, 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$method'.

PodsBot · 2024-10-01T03:55:44Z

components/Migrate-PHP/ui/wizard.php

+		<div class="pods-submittable-fields">
+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>
+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>
+			<?php echo PodsForm::field( 'method', $method, 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/ui/wizard.php

Line 103 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 103 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 103 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 103 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 103 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 103 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in 977d854

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in 977d854

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in 977d854

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 106 in 977d854

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 106 in 977d854

] );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data".

pods/components/Migrate-PHP/ui/wizard.php

Line 121 in 977d854

$data = $pod_pages;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data_name".

pods/components/Migrate-PHP/ui/wizard.php

Line 122 in 977d854

$data_name = 'pages';

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 140 in 977d854

$zebra = false;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$item".

pods/components/Migrate-PHP/ui/wizard.php

Line 142 in 977d854

foreach ( $data as $item ) {

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$checked".

pods/components/Migrate-PHP/ui/wizard.php

Line 143 in 977d854

$checked = true;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$class".

pods/components/Migrate-PHP/ui/wizard.php

Line 145 in 977d854

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 145 in 977d854

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 147 in 977d854

$zebra = ( ! $zebra );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 977d854

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 977d854

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 977d854

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 977d854

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 154 in 977d854

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 154 in 977d854

] );

[phpcs] _{reported by reviewdog 🐶}
Universal.NamingConventions.NoReservedKeywordParameterNames.objectFound
It is recommended not to use reserved keyword "object" as function parameter name. Found: $object

pods/components/Pages.php

Line 1058 in 977d854

public static function object_to_page( Page $object ): array {

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.InlineComment.InvalidEndChar
Inline comments must end in full-stops, exclamation marks, or question marks

pods/components/Pages.php

Line 1066 in 977d854

// phpcode is deprecated

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1071 in 977d854

    
           'admin_only'              => (boolean) get_post_meta( $id, 'admin_only', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1072 in 977d854

    
           'restrict_role'           => (boolean) get_post_meta( $id, 'restrict_role', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1073 in 977d854

    
           'restrict_capability'     => (boolean) get_post_meta( $id, 'restrict_capability', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1076 in 977d854

    
           'restrict_redirect'       => (boolean) get_post_meta( $id, 'restrict_redirect', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1077 in 977d854

    
           'restrict_redirect_login' => (boolean) get_post_meta( $id, 'restrict_redirect_login', true ),

[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

pods/components/Templates/Templates.php

Line 714 in 977d854

$default_templates = array(

PodsBot · 2024-10-01T05:05:19Z

components/Migrate-PHP/ui/wizard.php

+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>
+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>
+			<?php echo PodsForm::field( 'method', $method, 'hidden' ); ?>
+			<?php echo PodsForm::field( '_wpnonce', wp_create_nonce( 'pods-component-' . $component . '-' . $method ), 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

PodsBot · 2024-10-01T05:05:19Z

components/Migrate-PHP/ui/wizard.php

+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>
+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>
+			<?php echo PodsForm::field( 'method', $method, 'hidden' ); ?>
+			<?php echo PodsForm::field( '_wpnonce', wp_create_nonce( 'pods-component-' . $component . '-' . $method ), 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'wp_create_nonce'.

PodsBot · 2024-10-01T05:05:19Z

components/Migrate-PHP/ui/wizard.php

+			<?php echo PodsForm::field( 'action', 'pods_admin_components', 'hidden' ); ?>
+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>
+			<?php echo PodsForm::field( 'method', $method, 'hidden' ); ?>
+			<?php echo PodsForm::field( '_wpnonce', wp_create_nonce( 'pods-component-' . $component . '-' . $method ), 'hidden' ); ?>


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

PodsBot · 2024-10-01T05:05:19Z

components/Migrate-PHP/ui/wizard.php

+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>
+			<?php echo PodsForm::field( 'method', $method, 'hidden' ); ?>
+			<?php echo PodsForm::field( '_wpnonce', wp_create_nonce( 'pods-component-' . $component . '-' . $method ), 'hidden' ); ?>
+			<?php echo PodsForm::field( 'cleanup', 0, 'hidden', array( 'attributes' => array( 'id' => 'pods_cleanup' ) ) ); ?>


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

PodsBot · 2024-10-01T05:05:19Z

components/Migrate-PHP/ui/wizard.php

+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>
+			<?php echo PodsForm::field( 'method', $method, 'hidden' ); ?>
+			<?php echo PodsForm::field( '_wpnonce', wp_create_nonce( 'pods-component-' . $component . '-' . $method ), 'hidden' ); ?>
+			<?php echo PodsForm::field( 'cleanup', 0, 'hidden', array( 'attributes' => array( 'id' => 'pods_cleanup' ) ) ); ?>


[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

PodsBot · 2024-10-01T05:05:19Z

components/Migrate-PHP/ui/wizard.php

+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>
+			<?php echo PodsForm::field( 'method', $method, 'hidden' ); ?>
+			<?php echo PodsForm::field( '_wpnonce', wp_create_nonce( 'pods-component-' . $component . '-' . $method ), 'hidden' ); ?>
+			<?php echo PodsForm::field( 'cleanup', 0, 'hidden', array( 'attributes' => array( 'id' => 'pods_cleanup' ) ) ); ?>


[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

PodsBot · 2024-10-01T05:05:19Z

components/Migrate-PHP/ui/wizard.php

+			<?php echo PodsForm::field( 'component', $component, 'hidden' ); ?>
+			<?php echo PodsForm::field( 'method', $method, 'hidden' ); ?>
+			<?php echo PodsForm::field( '_wpnonce', wp_create_nonce( 'pods-component-' . $component . '-' . $method ), 'hidden' ); ?>
+			<?php echo PodsForm::field( 'cleanup', 0, 'hidden', array( 'attributes' => array( 'id' => 'pods_cleanup' ) ) ); ?>


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 099ba47

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 099ba47

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 099ba47

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 099ba47

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 151 in 099ba47

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 099ba47

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 099ba47

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 099ba47

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 154 in 099ba47

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 154 in 099ba47

] );

[phpcs] _{reported by reviewdog 🐶}
Universal.NamingConventions.NoReservedKeywordParameterNames.objectFound
It is recommended not to use reserved keyword "object" as function parameter name. Found: $object

pods/components/Pages.php

Line 1058 in 099ba47

public static function object_to_page( Page $object ): array {

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.InlineComment.InvalidEndChar
Inline comments must end in full-stops, exclamation marks, or question marks

pods/components/Pages.php

Line 1066 in 099ba47

// phpcode is deprecated

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1071 in 099ba47

    
           'admin_only'              => (boolean) get_post_meta( $id, 'admin_only', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1072 in 099ba47

    
           'restrict_role'           => (boolean) get_post_meta( $id, 'restrict_role', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1073 in 099ba47

    
           'restrict_capability'     => (boolean) get_post_meta( $id, 'restrict_capability', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1076 in 099ba47

    
           'restrict_redirect'       => (boolean) get_post_meta( $id, 'restrict_redirect', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1077 in 099ba47

    
           'restrict_redirect_login' => (boolean) get_post_meta( $id, 'restrict_redirect_login', true ),

[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

pods/components/Templates/Templates.php

Line 714 in 099ba47

$default_templates = array(

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1110 in cc321eb

    
           'restrict_redirect'       => (boolean) get_post_meta( $id, 'restrict_redirect', true ),

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Pages.php

Line 1111 in cc321eb

    
           'restrict_redirect_login' => (boolean) get_post_meta( $id, 'restrict_redirect_login', true ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not 'self'

pods/components/Templates/Templates.php

Line 645 in cc321eb

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'self'.

pods/components/Templates/Templates.php

Line 645 in cc321eb

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$obj'

pods/components/Templates/Templates.php

Line 645 in cc321eb

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$obj'.

pods/components/Templates/Templates.php

Line 645 in cc321eb

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not 'true'

pods/components/Templates/Templates.php

Line 645 in cc321eb

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not 'self'

pods/components/Templates/Templates.php

Line 657 in cc321eb

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'self'.

pods/components/Templates/Templates.php

Line 657 in cc321eb

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$obj'

pods/components/Templates/Templates.php

Line 657 in cc321eb

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$obj'.

pods/components/Templates/Templates.php

Line 657 in cc321eb

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not 'true'

pods/components/Templates/Templates.php

Line 657 in cc321eb

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$template_output'

pods/components/Templates/Templates.php

Line 692 in cc321eb

echo $template_output;

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$template_output'.

pods/components/Templates/Templates.php

Line 692 in cc321eb

echo $template_output;

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$template_output'

pods/components/Templates/Templates.php

Line 710 in cc321eb

echo $template_output;

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$template_output'.

pods/components/Templates/Templates.php

Line 710 in cc321eb

echo $template_output;

[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

pods/components/Templates/Templates.php

Line 737 in cc321eb

$default_templates = array(

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Templates/Templates.php

Line 777 in cc321eb

$data = get_file_data( $file_path, [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Templates/Templates.php

Line 777 in cc321eb

$data = get_file_data( $file_path, [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned
Array double arrow not aligned correctly; expected 1 space(s) between "'PodTemplate'" and double arrow, but found 2.

pods/components/Templates/Templates.php

Line 778 in cc321eb

'PodTemplate' => 'Pod Template',

[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned
Array double arrow not aligned correctly; expected 3 space(s) between "'MagicTags'" and double arrow, but found 1.

pods/components/Templates/Templates.php

Line 779 in cc321eb

'MagicTags' => 'Magic Tags',

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Templates/Templates.php

Line 780 in cc321eb

] );

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.ParamCommentFullStop
Parameter comment must end with a full stop

pods/components/Templates/Templates.php

Line 793 in cc321eb

* @param string $code The template string to parse

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.ParamCommentFullStop
Parameter comment must end with a full stop

pods/components/Templates/Templates.php

Line 794 in cc321eb

* @param object $obj The Pods object

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'admin_url'.

pods/components/Templates/includes/element-view_template.php

Line 38 in cc321eb

admin_url( 'admin.php?page=pods-components' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Templates/includes/element-view_template.php

Line 38 in cc321eb

admin_url( 'admin.php?page=pods-components' ),

PodsBot · 2024-10-03T03:24:31Z

components/Pages.php

+				'name'         => 'precode_notice',
+				'type'         => 'html',
+				'html_content' => sprintf(
+			'


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.Indent
Multi-line function call not indented correctly; expected 20 spaces but found 12

PodsBot · 2024-10-03T03:24:31Z

components/Pages.php

+				'name'         => 'precode_notice',
+				'type'         => 'html',
+				'html_content' => sprintf(
+			'


[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.ArrayIndentation.MultiLineArrayItemNotAligned
Multi-line array item not aligned correctly; expected 16 spaces, but found 12

PodsBot · 2024-10-03T03:24:32Z

components/Pages.php

@@ -653,6 +696,93 @@

 		add_action( 'admin_enqueue_scripts', array( $this, 'admin_assets' ), 21 );
 		add_filter( 'enter_title_here', array( $this, 'set_title_text' ), 10, 2 );
+
+		$page_code = get_the_content();
+		$pre_code = get_post_meta( get_the_ID(), 'precode', true );


[phpcs] _{reported by reviewdog 🐶}
Generic.Formatting.MultipleStatementAlignment.NotSameWarning
Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

PodsBot · 2024-10-03T03:24:32Z

components/Pages.php

+		$page_code = get_the_content();
+		$pre_code = get_post_meta( get_the_ID(), 'precode', true );
+
+		$has_php = false !== strpos( $page_code, '<?' );


[phpcs] _{reported by reviewdog 🐶}
Generic.Formatting.MultipleStatementAlignment.NotSameWarning
Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space

PodsBot · 2024-10-03T03:24:32Z

components/Pages.php

+	 *
+	 * @return array The Pod Page data array.
+	 */
+	public static function object_to_page( Page $object ): array {


[phpcs] _{reported by reviewdog 🐶}
Universal.NamingConventions.NoReservedKeywordParameterNames.objectFound
It is recommended not to use reserved keyword "object" as function parameter name. Found: $object

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$obj'

pods/components/Templates/Templates.php

Line 657 in 8a9a3dc

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$obj'.

pods/components/Templates/Templates.php

Line 657 in 8a9a3dc

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not 'true'

pods/components/Templates/Templates.php

Line 657 in 8a9a3dc

echo self::do_template( $code, $obj, true );

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$template_output'

pods/components/Templates/Templates.php

Line 692 in 8a9a3dc

echo $template_output;

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$template_output'.

pods/components/Templates/Templates.php

Line 692 in 8a9a3dc

echo $template_output;

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$template_output'

pods/components/Templates/Templates.php

Line 710 in 8a9a3dc

echo $template_output;

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$template_output'.

pods/components/Templates/Templates.php

Line 710 in 8a9a3dc

echo $template_output;

[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

pods/components/Templates/Templates.php

Line 737 in 8a9a3dc

$default_templates = array(

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Templates/Templates.php

Line 777 in 8a9a3dc

$data = get_file_data( $file_path, [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Templates/Templates.php

Line 777 in 8a9a3dc

$data = get_file_data( $file_path, [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned
Array double arrow not aligned correctly; expected 1 space(s) between "'PodTemplate'" and double arrow, but found 2.

pods/components/Templates/Templates.php

Line 778 in 8a9a3dc

'PodTemplate' => 'Pod Template',

[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned
Array double arrow not aligned correctly; expected 3 space(s) between "'MagicTags'" and double arrow, but found 1.

pods/components/Templates/Templates.php

Line 779 in 8a9a3dc

'MagicTags' => 'Magic Tags',

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Templates/Templates.php

Line 780 in 8a9a3dc

] );

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.ParamCommentFullStop
Parameter comment must end with a full stop

pods/components/Templates/Templates.php

Line 793 in 8a9a3dc

* @param string $code The template string to parse

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.ParamCommentFullStop
Parameter comment must end with a full stop

pods/components/Templates/Templates.php

Line 794 in 8a9a3dc

* @param object $obj The Pods object

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'admin_url'.

pods/components/Templates/includes/element-view_template.php

Line 38 in 8a9a3dc

admin_url( 'admin.php?page=pods-components' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Templates/includes/element-view_template.php

Line 38 in 8a9a3dc

admin_url( 'admin.php?page=pods-components' ),

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

pods/components/Templates/Templates.php

Line 743 in e66ddd2

$default_templates = array(

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Templates/Templates.php

Line 783 in e66ddd2

$data = get_file_data( $file_path, [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Templates/Templates.php

Line 783 in e66ddd2

$data = get_file_data( $file_path, [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned
Array double arrow not aligned correctly; expected 1 space(s) between "'PodTemplate'" and double arrow, but found 2.

pods/components/Templates/Templates.php

Line 784 in e66ddd2

'PodTemplate' => 'Pod Template',

[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned
Array double arrow not aligned correctly; expected 3 space(s) between "'MagicTags'" and double arrow, but found 1.

pods/components/Templates/Templates.php

Line 785 in e66ddd2

'MagicTags' => 'Magic Tags',

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Templates/Templates.php

Line 786 in e66ddd2

] );

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.ParamCommentFullStop
Parameter comment must end with a full stop

pods/components/Templates/Templates.php

Line 799 in e66ddd2

* @param string $code The template string to parse

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.ParamCommentFullStop
Parameter comment must end with a full stop

pods/components/Templates/Templates.php

Line 800 in e66ddd2

* @param object $obj The Pods object

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'admin_url'.

pods/components/Templates/includes/element-view_template.php

Line 38 in e66ddd2

admin_url( 'admin.php?page=pods-components' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Templates/includes/element-view_template.php

Line 38 in e66ddd2

admin_url( 'admin.php?page=pods-components' ),

PodsBot · 2024-10-09T18:09:39Z

components/Pages.php

+			'uri'           => $object->get_label(),
+			'code'          => $object->get_description(),
+			'phpcode'       => $object->get_description(),
+			// phpcode is deprecated


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.InlineComment.InvalidEndChar
Inline comments must end in full-stops, exclamation marks, or question marks

PodsBot · 2024-10-09T18:09:39Z

components/Pages.php

+			'page_template' => get_post_meta( $id, 'page_template', true ),
+			'title'         => get_post_meta( $id, 'page_title', true ),
+			'options'       => [
+				'admin_only'              => (boolean) get_post_meta( $id, 'admin_only', true ),


[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

PodsBot · 2024-10-09T18:09:39Z

components/Pages.php

+			'title'         => get_post_meta( $id, 'page_title', true ),
+			'options'       => [
+				'admin_only'              => (boolean) get_post_meta( $id, 'admin_only', true ),
+				'restrict_role'           => (boolean) get_post_meta( $id, 'restrict_role', true ),


[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

PodsBot · 2024-10-09T18:09:39Z

components/Pages.php

+			'options'       => [
+				'admin_only'              => (boolean) get_post_meta( $id, 'admin_only', true ),
+				'restrict_role'           => (boolean) get_post_meta( $id, 'restrict_role', true ),
+				'restrict_capability'     => (boolean) get_post_meta( $id, 'restrict_capability', true ),


[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

PodsBot · 2024-10-09T18:09:39Z

components/Pages.php

+				'restrict_capability'     => (boolean) get_post_meta( $id, 'restrict_capability', true ),
+				'roles_allowed'           => get_post_meta( $id, 'roles_allowed', true ),
+				'capability_allowed'      => get_post_meta( $id, 'capability_allowed', true ),
+				'restrict_redirect'       => (boolean) get_post_meta( $id, 'restrict_redirect', true ),


[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

PodsBot · 2024-10-09T18:09:41Z

components/Templates/Templates.php

@@ -641,17 +667,13 @@
 						&& ! pods_access_bypass_private_post( $info )
 					)
 				) {
-					echo self::do_template( $code, $obj );
+					echo self::do_template( $code, $obj, true );


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not 'true'

PodsBot · 2024-10-09T18:09:41Z

components/Templates/Templates.php

+						$template_output = self::do_template( $template_output, $obj );
+					}
+
+					echo $template_output;


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$template_output'

PodsBot · 2024-10-09T18:09:41Z

components/Templates/Templates.php

+						$template_output = self::do_template( $template_output, $obj );
+					}
+
+					echo $template_output;


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$template_output'.

PodsBot · 2024-10-09T18:09:41Z

components/Templates/Templates.php

+						$template_output = self::do_template( $template_output, $obj );
+					}
+
+					echo $template_output;


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$template_output'

PodsBot · 2024-10-09T18:09:41Z

components/Templates/Templates.php

+						$template_output = self::do_template( $template_output, $obj );
+					}
+
+					echo $template_output;


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$template_output'.

…Pods object is not returned

…s null

…e file exists

PodsBot · 2024-11-17T00:14:48Z

components/Templates/Templates.php

+	public static function get_templates_for_pod_template( $template, $obj = null ): array {
+		$template_name = trim( preg_replace( '/[^a-zA-Z0-9_\-\/]/', '', $template->get_name() ), ' /-' );
+
+		$default_templates = array(


[phpcs] _{reported by reviewdog 🐶}
Generic.Arrays.DisallowLongArraySyntax.Found
Short array syntax must be used to define arrays

PodsBot · 2024-11-17T00:14:49Z

components/Templates/Templates.php

+				continue;
+			}
+
+			$data = get_file_data( $file_path, [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

PodsBot · 2024-11-17T00:14:49Z

components/Templates/Templates.php

+				continue;
+			}
+
+			$data = get_file_data( $file_path, [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-11-17T00:14:49Z

components/Templates/Templates.php

+			}
+
+			$data = get_file_data( $file_path, [
+				'PodTemplate'  => 'Pod Template',


[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned
Array double arrow not aligned correctly; expected 1 space(s) between "'PodTemplate'" and double arrow, but found 2.

PodsBot · 2024-11-17T00:14:49Z

components/Templates/Templates.php

+
+			$data = get_file_data( $file_path, [
+				'PodTemplate'  => 'Pod Template',
+				'MagicTags' => 'Magic Tags',


[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned
Array double arrow not aligned correctly; expected 3 space(s) between "'MagicTags'" and double arrow, but found 1.

PodsBot · 2024-11-17T00:14:49Z

components/Templates/Templates.php

+			$data = get_file_data( $file_path, [
+				'PodTemplate'  => 'Pod Template',
+				'MagicTags' => 'Magic Tags',
+			] );


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

PodsBot · 2024-11-17T00:14:49Z

components/Templates/Templates.php

 	/**
 	 * Parse a template string
 	 *
-	 * @param string $code The template string to parse
-	 * @param object $obj  The Pods object
+	 * @param string $code        The template string to parse


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.ParamCommentFullStop
Parameter comment must end with a full stop

PodsBot · 2024-11-17T00:14:49Z

components/Templates/Templates.php

-	 * @param string $code The template string to parse
-	 * @param object $obj  The Pods object
+	 * @param string $code        The template string to parse
+	 * @param object $obj         The Pods object


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.ParamCommentFullStop
Parameter comment must end with a full stop

PodsBot · 2024-11-17T00:14:49Z

components/Templates/includes/element-view_template.php

+		esc_html__( 'PHP detected, this feature is deprecated', 'pods' ),
+		'https://docs.pods.io/displaying-pods/pod-template-hierarchy-for-themes/',
+		esc_html__( 'Read more about file-based templates', 'pods' ),
+		admin_url( 'admin.php?page=pods-components' ),


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'admin_url'.

PodsBot · 2024-11-17T00:14:49Z

components/Templates/includes/element-view_template.php

+		esc_html__( 'PHP detected, this feature is deprecated', 'pods' ),
+		'https://docs.pods.io/displaying-pods/pod-template-hierarchy-for-themes/',
+		esc_html__( 'Read more about file-based templates', 'pods' ),
+		admin_url( 'admin.php?page=pods-components' ),


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

…ry in podsrel

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

pods/components/Migrate-PHP/Migrate-PHP.php

Line 154 in c445492

    
           if ( true === (boolean) $checked && isset( $pod_pages_available_to_migrate[ (int) $object_id ] ) ) {

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.Missing
Missing doc comment for function setup_file_path()

pods/components/Migrate-PHP/Migrate-PHP.php

Line 219 in c445492

private function setup_file_path( $file_path ) {

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.Missing
Missing doc comment for function migrate_template()

pods/components/Migrate-PHP/Migrate-PHP.php

Line 250 in c445492

private function migrate_template( $object_id, bool $cleanup ) {

[phpcs] _{reported by reviewdog 🐶}
WordPress.WP.AlternativeFunctions.json_encode_json_encode
json_encode() is discouraged. Use wp_json_encode() instead.

pods/components/Migrate-PHP/Migrate-PHP.php

Line 270 in c445492

    
           pods_error( sprintf( esc_html__( 'Unable to detect the file path: %s', 'pods' ), json_encode( $files, JSON_PRETTY_PRINT ) ) );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 282 in c445492

$extra_headers = <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 288 in c445492

$contents = <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/Migrate-PHP.php

Line 306 in c445492

$api->save_template( [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned
Array double arrow not aligned correctly; expected 1 space(s) between "'name'" and double arrow, but found 3.

pods/components/Migrate-PHP/Migrate-PHP.php

Line 308 in c445492

'name' => $object->get_label(),

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/Migrate-PHP.php

Line 310 in c445492

] );

[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.Missing
Missing doc comment for function migrate_page()

pods/components/Migrate-PHP/Migrate-PHP.php

Line 316 in c445492

private function migrate_page( $object_id, bool $cleanup ) {

[phpcs] _{reported by reviewdog 🐶}
WordPress.WP.AlternativeFunctions.json_encode_json_encode
json_encode() is discouraged. Use wp_json_encode() instead.

pods/components/Migrate-PHP/Migrate-PHP.php

Line 337 in c445492

    
           pods_error( sprintf( esc_html__( 'Unable to detect the file path: %s', 'pods' ), json_encode( $files, JSON_PRETTY_PRINT ) ) );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 355 in c445492

$precode_template = "\n" . <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/Migrate-PHP.php

Line 371 in c445492

    
           			$template_code = "?>\n" . $template_code . ( ! $has_page_template ? '' : "\n<?php" );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 388 in c445492

$template_code = $start_tag . <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 402 in c445492

$extra_notes .= "\n" . <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
Generic.Formatting.MultipleStatementAlignment.NotSameWarning
Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space

pods/components/Migrate-PHP/Migrate-PHP.php

Line 409 in c445492

$file_path = $file_path_for_content;

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 410 in c445492

$extra_notes .= "\n" . <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 417 in c445492

$extra_headers = "\n" . <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 422 in c445492

$contents = <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/Migrate-PHP.php

Line 446 in c445492

$api->save_page( [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/Migrate-PHP.php

Line 451 in c445492

] );

[phpcs] _{reported by reviewdog 🐶}
PSR2.Classes.ClassDeclaration.CloseBraceAfterBody
The closing brace for the class must go on the next line after the body

pods/components/Migrate-PHP/Migrate-PHP.php

Line 457 in c445492

}

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data".

pods/components/Migrate-PHP/ui/wizard.php

Line 74 in c445492

$data = $pod_templates;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data_name".

pods/components/Migrate-PHP/ui/wizard.php

Line 75 in c445492

$data_name = 'templates';

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 93 in c445492

$zebra = false;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$item".

pods/components/Migrate-PHP/ui/wizard.php

Line 95 in c445492

foreach ( $data as $item ) {

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$checked".

pods/components/Migrate-PHP/ui/wizard.php

Line 96 in c445492

$checked = true;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$class".

pods/components/Migrate-PHP/ui/wizard.php

Line 98 in c445492

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 98 in c445492

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 100 in c445492

$zebra = ( ! $zebra );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 105 in c445492

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 105 in c445492

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 105 in c445492

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 107 in c445492

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 107 in c445492

] );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data".

pods/components/Migrate-PHP/ui/wizard.php

Line 122 in c445492

$data = $pod_pages;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data_name".

pods/components/Migrate-PHP/ui/wizard.php

Line 123 in c445492

$data_name = 'pages';

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 141 in c445492

$zebra = false;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$item".

pods/components/Migrate-PHP/ui/wizard.php

Line 143 in c445492

foreach ( $data as $item ) {

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$checked".

pods/components/Migrate-PHP/ui/wizard.php

Line 144 in c445492

$checked = true;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$class".

pods/components/Migrate-PHP/ui/wizard.php

Line 146 in c445492

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 146 in c445492

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 148 in c445492

$zebra = ( ! $zebra );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in c445492

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 153 in c445492

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 153 in c445492

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 153 in c445492

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 155 in c445492

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 155 in c445492

] );

PodsBot · 2024-11-17T05:34:00Z

components/Migrate-PHP/Migrate-PHP.php

+		if ( class_exists( 'Pods_Templates' ) ) {
+			$pod_templates = array_filter(
+				$api->load_templates(),
+				static function( $object ) {


[phpcs] _{reported by reviewdog 🐶}
Squiz.Functions.MultiLineFunctionDeclaration.SpaceAfterFunction
Expected 1 space after FUNCTION keyword; 0 found

PodsBot · 2024-11-17T05:34:00Z

components/Migrate-PHP/Migrate-PHP.php

+		if ( class_exists( 'Pods_Templates' ) ) {
+			$pod_templates = array_filter(
+				$api->load_templates(),
+				static function( $object ) {


[phpcs] _{reported by reviewdog 🐶}
Universal.NamingConventions.NoReservedKeywordParameterNames.objectFound
It is recommended not to use reserved keyword "object" as function parameter name. Found: $object

PodsBot · 2024-11-17T05:34:00Z

components/Migrate-PHP/Migrate-PHP.php

+		if ( class_exists( 'Pods_Pages' ) ) {
+			$pod_pages = array_filter(
+				$api->load_pages(),
+				static function( $object ) {


[phpcs] _{reported by reviewdog 🐶}
Squiz.Functions.MultiLineFunctionDeclaration.SpaceAfterFunction
Expected 1 space after FUNCTION keyword; 0 found

PodsBot · 2024-11-17T05:34:00Z

components/Migrate-PHP/Migrate-PHP.php

+		if ( class_exists( 'Pods_Pages' ) ) {
+			$pod_pages = array_filter(
+				$api->load_pages(),
+				static function( $object ) {


[phpcs] _{reported by reviewdog 🐶}
Universal.NamingConventions.NoReservedKeywordParameterNames.objectFound
It is recommended not to use reserved keyword "object" as function parameter name. Found: $object

PodsBot · 2024-11-17T05:34:00Z

components/Migrate-PHP/Migrate-PHP.php

+		}
+
+		// Rekey the objects by ID.
+		$pod_templates = array_combine( array_map( static function( $object ) {


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

PodsBot · 2024-11-17T05:34:01Z

components/Migrate-PHP/Migrate-PHP.php

+
+		$pod_pages = array_combine( array_map( static function( $object ) {
+			return $object->get_id();
+		}, $pod_pages ), $pod_pages );


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

PodsBot · 2024-11-17T05:34:01Z

components/Migrate-PHP/Migrate-PHP.php

+		return compact( 'pod_templates', 'pod_pages' );
+	}
+
+	/**


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.MissingParamTag
Doc comment for parameter "$options" missing

PodsBot · 2024-11-17T05:34:01Z

components/Migrate-PHP/Migrate-PHP.php

+		return compact( 'pod_templates', 'pod_pages' );
+	}
+
+	/**


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.MissingParamTag
Doc comment for parameter "$component" missing

PodsBot · 2024-11-17T05:34:01Z

components/Migrate-PHP/Migrate-PHP.php

+	/**
+	 * Show the Admin
+	 *
+	 * @param $options


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.MissingParamType
Missing parameter type

PodsBot · 2024-11-17T05:34:01Z

components/Migrate-PHP/Migrate-PHP.php

+	 * Show the Admin
+	 *
+	 * @param $options
+	 * @param $component


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.MissingParamType
Missing parameter type

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 410 in 4ccfb0e

$extra_notes .= "\n" . <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 417 in 4ccfb0e

$extra_headers = "\n" . <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.Heredoc.NotAllowed
Use of heredoc and nowdoc syntax ("<<<") is not allowed; use standard strings or inline HTML instead

pods/components/Migrate-PHP/Migrate-PHP.php

Line 422 in 4ccfb0e

$contents = <<<PHPTEMPLATE

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/Migrate-PHP.php

Line 446 in 4ccfb0e

$api->save_page( [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/Migrate-PHP.php

Line 451 in 4ccfb0e

] );

[phpcs] _{reported by reviewdog 🐶}
PSR2.Classes.ClassDeclaration.CloseBraceAfterBody
The closing brace for the class must go on the next line after the body

pods/components/Migrate-PHP/Migrate-PHP.php

Line 457 in 4ccfb0e

}

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data".

pods/components/Migrate-PHP/ui/wizard.php

Line 74 in 4ccfb0e

$data = $pod_templates;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data_name".

pods/components/Migrate-PHP/ui/wizard.php

Line 75 in 4ccfb0e

$data_name = 'templates';

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 93 in 4ccfb0e

$zebra = false;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$item".

pods/components/Migrate-PHP/ui/wizard.php

Line 95 in 4ccfb0e

foreach ( $data as $item ) {

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$checked".

pods/components/Migrate-PHP/ui/wizard.php

Line 96 in 4ccfb0e

$checked = true;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$class".

pods/components/Migrate-PHP/ui/wizard.php

Line 98 in 4ccfb0e

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 98 in 4ccfb0e

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 100 in 4ccfb0e

$zebra = ( ! $zebra );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 105 in 4ccfb0e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 105 in 4ccfb0e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 105 in 4ccfb0e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 107 in 4ccfb0e

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 107 in 4ccfb0e

] );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data".

pods/components/Migrate-PHP/ui/wizard.php

Line 122 in 4ccfb0e

$data = $pod_pages;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data_name".

pods/components/Migrate-PHP/ui/wizard.php

Line 123 in 4ccfb0e

$data_name = 'pages';

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 141 in 4ccfb0e

$zebra = false;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$item".

pods/components/Migrate-PHP/ui/wizard.php

Line 143 in 4ccfb0e

foreach ( $data as $item ) {

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$checked".

pods/components/Migrate-PHP/ui/wizard.php

Line 144 in 4ccfb0e

$checked = true;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$class".

pods/components/Migrate-PHP/ui/wizard.php

Line 146 in 4ccfb0e

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 146 in 4ccfb0e

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 148 in 4ccfb0e

$zebra = ( ! $zebra );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in 4ccfb0e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 153 in 4ccfb0e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 153 in 4ccfb0e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 153 in 4ccfb0e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 155 in 4ccfb0e

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 155 in 4ccfb0e

] );

[phpcs] _{reported by reviewdog 🐶}
SlevomatCodingStandard.Variables.UselessVariable.UselessVariable
Useless variable $plugin.

pods/src/Pods/Wisdom_Tracker.php

Line 378 in 4ccfb0e

$plugin = get_plugin_data( $this->plugin_file, true, false );

PodsBot · 2024-11-17T05:36:36Z

classes/PodsAPI.php

+				$sister_field = $this->load_field( [ 'id' => $sister_id ] );
+
+				if ( $sister_field ) {
+					pods_query( '


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

PodsBot · 2024-11-17T05:36:36Z

classes/PodsAPI.php

+							`related_pod_id` = %d,
+							`related_field_id` = %d
+						WHERE `field_id` = %d
+					', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-11-17T05:36:36Z

classes/PodsAPI.php

+							`related_field_id` = %d
+						WHERE `field_id` = %d
+					', [
+						$pod['id'],


[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.ArrayIndentation.ItemNotAligned
Array item not aligned correctly; expected 4 spaces but found 24

PodsBot · 2024-11-17T05:36:36Z

classes/PodsAPI.php

+						WHERE `field_id` = %d
+					', [
+						$pod['id'],
+						$params->id,


[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.ArrayIndentation.ItemNotAligned
Array item not aligned correctly; expected 4 spaces but found 24

PodsBot · 2024-11-17T05:36:37Z

classes/PodsAPI.php

+					', [
+						$pod['id'],
+						$params->id,
+						$sister_id,


[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.ArrayIndentation.ItemNotAligned
Array item not aligned correctly; expected 4 spaces but found 24

PodsBot · 2024-11-17T05:36:37Z

classes/PodsAPI.php

+							`related_field_id` = %d
+						WHERE `field_id` = %d
+					', [
+						$sister_field->get_parent_id(),


[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.ArrayIndentation.ItemNotAligned
Array item not aligned correctly; expected 4 spaces but found 24

PodsBot · 2024-11-17T05:36:37Z

classes/PodsAPI.php

+						WHERE `field_id` = %d
+					', [
+						$sister_field->get_parent_id(),
+						$sister_id,


[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.ArrayIndentation.ItemNotAligned
Array item not aligned correctly; expected 4 spaces but found 24

PodsBot · 2024-11-17T05:36:37Z

classes/PodsAPI.php

+					', [
+						$sister_field->get_parent_id(),
+						$sister_id,
+						$params->id,


[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.ArrayIndentation.ItemNotAligned
Array item not aligned correctly; expected 4 spaces but found 24

PodsBot · 2024-11-17T05:36:37Z

classes/PodsAPI.php

+						$sister_field->get_parent_id(),
+						$sister_id,
+						$params->id,
+					] );


[phpcs] _{reported by reviewdog 🐶}
WordPress.Arrays.ArrayIndentation.CloseBraceNotAligned
Array closer not aligned correctly; expected 0 space(s) but found 20

PodsBot · 2024-11-17T05:36:37Z

classes/PodsAPI.php

+						$sister_field->get_parent_id(),
+						$sister_id,
+						$params->id,
+					] );


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$checked".

pods/components/Migrate-PHP/ui/wizard.php

Line 102 in d91a00e

$checked = true;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$class".

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in d91a00e

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 104 in d91a00e

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 106 in d91a00e

$zebra = ( ! $zebra );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Migrate-PHP/ui/wizard.php

Line 110 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/ui/wizard.php

Line 110 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 110 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 110 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 110 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 110 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 110 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 111 in d91a00e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 111 in d91a00e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 111 in d91a00e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 113 in d91a00e

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 113 in d91a00e

] );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data".

pods/components/Migrate-PHP/ui/wizard.php

Line 128 in d91a00e

$data = $pod_pages;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data_name".

pods/components/Migrate-PHP/ui/wizard.php

Line 129 in d91a00e

$data_name = 'pages';

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 147 in d91a00e

$zebra = false;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$item".

pods/components/Migrate-PHP/ui/wizard.php

Line 149 in d91a00e

foreach ( $data as $item ) {

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$checked".

pods/components/Migrate-PHP/ui/wizard.php

Line 150 in d91a00e

$checked = true;

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$class".

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in d91a00e

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 152 in d91a00e

$class = ( $zebra ? 'even' : 'odd' );

[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

pods/components/Migrate-PHP/ui/wizard.php

Line 154 in d91a00e

$zebra = ( ! $zebra );

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

pods/components/Migrate-PHP/ui/wizard.php

Line 158 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

pods/components/Migrate-PHP/ui/wizard.php

Line 158 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 158 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

pods/components/Migrate-PHP/ui/wizard.php

Line 158 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 158 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 158 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 158 in d91a00e

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 159 in d91a00e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 159 in d91a00e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 159 in d91a00e

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 161 in d91a00e

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 161 in d91a00e

] );

[phpcs] _{reported by reviewdog 🐶}
SlevomatCodingStandard.Variables.UselessVariable.UselessVariable
Useless variable $plugin.

pods/src/Pods/Wisdom_Tracker.php

Line 378 in d91a00e

$plugin = get_plugin_data( $this->plugin_file, true, false );

PodsBot · 2024-11-17T05:49:08Z

components/Migrate-PHP/Migrate-PHP.php

+
+		$has_objects_to_migrate = ! empty( $pod_templates ) || ! empty( $pod_pages );
+
+		// ajax_migrate


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.InlineComment.InvalidEndChar
Inline comments must end in full-stops, exclamation marks, or question marks

PodsBot · 2024-11-17T05:49:08Z

components/Migrate-PHP/Migrate-PHP.php

+		pods_view( __DIR__ . '/ui/wizard.php', compact( array_keys( get_defined_vars() ) ) );
+	}
+
+	/**


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.MissingParamTag
Doc comment for parameter "$params" missing

PodsBot · 2024-11-17T05:49:08Z

components/Migrate-PHP/Migrate-PHP.php

+	/**
+	 * Handle the Migration AJAX
+	 *
+	 * @param $params


[phpcs] _{reported by reviewdog 🐶}
Squiz.Commenting.FunctionComment.MissingParamType
Missing parameter type

PodsBot · 2024-11-17T05:49:08Z

components/Migrate-PHP/Migrate-PHP.php

+		$has_objects_to_migrate = ! empty( $pod_templates_selected ) || ! empty( $pod_pages_selected );
+
+		foreach ( $pod_templates_selected as $object_id => $checked ) {
+			if ( true === (boolean) $checked && isset( $pod_templates_available_to_migrate[ (int) $object_id ] ) ) {


[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

PodsBot · 2024-11-17T05:49:08Z

components/Migrate-PHP/Migrate-PHP.php

+		}
+
+		foreach ( $pod_pages_selected as $object_id => $checked ) {
+			if ( true === (boolean) $checked && isset( $pod_pages_available_to_migrate[ (int) $object_id ] ) ) {


[phpcs] _{reported by reviewdog 🐶}
PSR12.Keywords.ShortFormTypeKeywords.LongFound
Short form type keywords must be used. Found: (boolean)

PodsBot · 2024-11-17T05:49:09Z

components/Migrate-PHP/Migrate-PHP.php

+		return str_replace( ABSPATH, '', $file_path );
+	}
+
+}


[phpcs] _{reported by reviewdog 🐶}
PSR2.Classes.ClassDeclaration.CloseBraceAfterBody
The closing brace for the class must go on the next line after the body

PodsBot · 2024-11-17T05:49:09Z

components/Migrate-PHP/ui/wizard.php

+
+						<?php if ( ! empty( $pod_templates ) ) : ?>
+							<?php
+							$data      = $pod_templates;


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data".

PodsBot · 2024-11-17T05:49:09Z

components/Migrate-PHP/ui/wizard.php

+						<?php if ( ! empty( $pod_templates ) ) : ?>
+							<?php
+							$data      = $pod_templates;
+							$data_name = 'templates';


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$data_name".

PodsBot · 2024-11-17T05:49:09Z

components/Migrate-PHP/ui/wizard.php

+
+											<ul>
+												<?php
+												$zebra = false;


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

PodsBot · 2024-11-17T05:49:09Z

components/Migrate-PHP/ui/wizard.php

+												<?php
+												$zebra = false;
+
+												foreach ( $data as $item ) {


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$item".

PodsBot

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

phpcs

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

pods/components/Migrate-PHP/ui/wizard.php

Line 158 in 011a845

    
           echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 159 in 011a845

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

pods/components/Migrate-PHP/ui/wizard.php

Line 159 in 011a845

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

pods/components/Migrate-PHP/ui/wizard.php

Line 159 in 011a845

    
           'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),

[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

pods/components/Migrate-PHP/ui/wizard.php

Line 161 in 011a845

] );

[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

pods/components/Migrate-PHP/ui/wizard.php

Line 161 in 011a845

] );

[phpcs] _{reported by reviewdog 🐶}
SlevomatCodingStandard.Variables.UselessVariable.UselessVariable
Useless variable $plugin.

pods/src/Pods/Wisdom_Tracker.php

Line 378 in 011a845

$plugin = get_plugin_data( $this->plugin_file, true, false );

PodsBot · 2024-11-17T16:28:07Z

components/Migrate-PHP/ui/wizard.php

+												$zebra = false;
+
+												foreach ( $data as $item ) {
+													$checked = true;


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$checked".

PodsBot · 2024-11-17T16:28:07Z

components/Migrate-PHP/ui/wizard.php

+												foreach ( $data as $item ) {
+													$checked = true;
+
+													$class = ( $zebra ? 'even' : 'odd' );


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$class".

PodsBot · 2024-11-17T16:28:07Z

components/Migrate-PHP/ui/wizard.php

+												foreach ( $data as $item ) {
+													$checked = true;
+
+													$class = ( $zebra ? 'even' : 'odd' );


[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

PodsBot · 2024-11-17T16:28:08Z

components/Migrate-PHP/ui/wizard.php

+
+													$class = ( $zebra ? 'even' : 'odd' );
+
+													$zebra = ( ! $zebra );


[phpcs] _{reported by reviewdog 🐶}
WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound
Global variables defined by a theme/plugin should start with the theme/plugin prefix. Found: "$zebra".

PodsBot · 2024-11-17T16:28:08Z

components/Migrate-PHP/ui/wizard.php

+													?>
+													<li class="pods-zebra-<?php echo esc_attr( $class ); ?>">
+														<?php
+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'PodsForm'.

PodsBot · 2024-11-17T16:28:09Z

components/Migrate-PHP/ui/wizard.php

+													?>
+													<li class="pods-zebra-<?php echo esc_attr( $class ); ?>">
+														<?php
+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket
Opening parenthesis of a multi-line function call must be the last content on the line

PodsBot · 2024-11-17T16:28:09Z

components/Migrate-PHP/ui/wizard.php

+													?>
+													<li class="pods-zebra-<?php echo esc_attr( $class ); ?>">
+														<?php
+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['id']'.

PodsBot · 2024-11-17T16:28:09Z

components/Migrate-PHP/ui/wizard.php

+													?>
+													<li class="pods-zebra-<?php echo esc_attr( $class ); ?>">
+														<?php
+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$checked'.

PodsBot · 2024-11-17T16:28:10Z

components/Migrate-PHP/ui/wizard.php

+													?>
+													<li class="pods-zebra-<?php echo esc_attr( $class ); ?>">
+														<?php
+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-11-17T16:28:10Z

components/Migrate-PHP/ui/wizard.php

+													?>
+													<li class="pods-zebra-<?php echo esc_attr( $class ); ?>">
+														<?php
+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-11-17T16:30:32Z

components/Migrate-PHP/ui/wizard.php

+													?>
+													<li class="pods-zebra-<?php echo esc_attr( $class ); ?>">
+														<?php
+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.MultipleArguments
Only one argument is allowed per line in a multi-line function call

PodsBot · 2024-11-17T16:30:32Z

components/Migrate-PHP/ui/wizard.php

+													<li class="pods-zebra-<?php echo esc_attr( $class ); ?>">
+														<?php
+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [
+															'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['name']'.

PodsBot · 2024-11-17T16:30:32Z

components/Migrate-PHP/ui/wizard.php

+													<li class="pods-zebra-<?php echo esc_attr( $class ); ?>">
+														<?php
+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [
+															'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),


[phpcs] _{reported by reviewdog 🐶}
Squiz.PHP.DisallowInlineIf.Found
Inline IF statements are not allowed

PodsBot · 2024-11-17T16:30:32Z

components/Migrate-PHP/ui/wizard.php

+													<li class="pods-zebra-<?php echo esc_attr( $class ); ?>">
+														<?php
+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [
+															'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),


[phpcs] _{reported by reviewdog 🐶}
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$item['label']'.

PodsBot · 2024-11-17T16:30:32Z

components/Migrate-PHP/ui/wizard.php

+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [
+															'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),
+															'disable_dfv'       => true,
+														] );


[phpcs] _{reported by reviewdog 🐶}
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

PodsBot · 2024-11-17T16:30:32Z

components/Migrate-PHP/ui/wizard.php

+														echo PodsForm::field( $data_name . '[' . $item['id'] . ']', $checked, 'boolean', [
+															'boolean_yes_label' => $item['name'] . ( ! empty( $item['label'] ) ? ' (' . $item['label'] . ')' : '' ),
+															'disable_dfv'       => true,
+														] );


[phpcs] _{reported by reviewdog 🐶}
PEAR.Functions.FunctionCallSignature.CloseBracketLine
Closing parenthesis of a multi-line function call must be on a line by itself

PodsBot · 2024-11-17T16:30:32Z

src/Pods/Wisdom_Tracker.php

@@ -375,7 +375,7 @@ public function plugin_data() {
 			include ABSPATH . '/wp-admin/includes/plugin.php';
 		}
 		// Retrieve current plugin information
-		$plugin = get_plugin_data( $this->plugin_file );
+		$plugin = get_plugin_data( $this->plugin_file, true, false );


[phpcs] _{reported by reviewdog 🐶}
SlevomatCodingStandard.Variables.UselessVariable.UselessVariable
Useless variable $plugin.

PodsBot · 2024-11-17T17:24:23Z

Total number of active review comments per pull request has been reached and some comments might not appear as a result. Please resolve some issues to see more (commit-ID: a780c71).

PodsBot added the Type: Release label Aug 31, 2024

PodsBot assigned sc0ttkclark Aug 31, 2024

PodsBot commented Aug 31, 2024

View reviewed changes

sc0ttkclark marked this pull request as ready for review August 31, 2024 02:24

sc0ttkclark requested review from sc0ttkclark and JoryHogeveen as code owners August 31, 2024 02:24

sc0ttkclark marked this pull request as draft August 31, 2024 02:24

pods-framework deleted a comment from PodsBot Aug 31, 2024

PodsBot commented Aug 31, 2024

View reviewed changes

PodsBot commented Sep 10, 2024

View reviewed changes

PodsBot commented Sep 12, 2024

View reviewed changes

github-advanced-security bot found potential problems Sep 12, 2024

View reviewed changes

ui/js/jquery.pods.js Fixed Show fixed Hide fixed

ui/js/jquery.pods.js Fixed Show fixed Hide fixed

ui/js/jquery.pods.js Fixed Show fixed Hide fixed

github-advanced-security bot found potential problems Sep 13, 2024

View reviewed changes

ui/js/jquery.pods.js Fixed Show fixed Hide fixed

ui/js/jquery.pods.js Fixed Show fixed Hide fixed

ui/js/jquery.pods.js Dismissed Show dismissed Hide dismissed

ui/js/jquery.pods.js Dismissed Show dismissed Hide dismissed

PodsBot commented Sep 13, 2024

View reviewed changes

PodsBot commented Sep 14, 2024

View reviewed changes

sc0ttkclark marked this pull request as ready for review September 14, 2024 15:11

PodsBot commented Sep 27, 2024

View reviewed changes

PodsBot commented Oct 1, 2024

View reviewed changes

PodsBot commented Oct 3, 2024

View reviewed changes

PodsBot commented Oct 9, 2024

View reviewed changes

sc0ttkclark and others added 4 commits November 16, 2024 18:12

Set version to 3.2.8-a-1

84b1ff3

Support enabling or disabling wpautop for certain kinds of block content

3745288

Add fallback for $pod->field() and the find argument in case the …

d8259fb

…Pods object is not returned

Disable phpcs rule for brackets and phpstan fix

70a0f51

sc0ttkclark added 2 commits November 16, 2024 18:12

Fix php warnings with methods that expect string when template code i…

37672cd

…s null

Ensure template code (if set) is not run if the corresponding templat…

9dad299

…e file exists

sc0ttkclark force-pushed the release/3.2.8 branch from e66ddd2 to 9dad299 Compare November 17, 2024 00:13

PodsBot commented Nov 17, 2024

View reviewed changes

sc0ttkclark force-pushed the main branch from 63e5e5b to 26fc546 Compare November 17, 2024 00:24

JoryHogeveen and others added 2 commits November 16, 2024 18:24

Validate non empty value

5887ead

Auto rebuild assets

f5884cd

sc0ttkclark force-pushed the release/3.2.8 branch from 6b7a7b5 to f5884cd Compare November 17, 2024 00:24

sc0ttkclark and others added 7 commits November 16, 2024 21:17

Update NPM packages and migrate SCSS from import to use

ffba468

Auto rebuild assets

caa4ea7

Update wizard text, improve validation of migration for templates/pages

c445492

Start the changelog

403e416

Fix PHP notice for array key in some situations

b7595ca

Ensure sister ID association sets all of the related items as necessa…

9bdfaac

…ry in podsrel

Move load_plugin_textdomain usage into init and make it optional

c40e87f

PodsBot commented Nov 17, 2024

View reviewed changes

When getting plugin data, don’t make translatable

4ccfb0e

PodsBot commented Nov 17, 2024

View reviewed changes

sc0ttkclark added 2 commits November 16, 2024 23:46

Show text when there are no objects to migrate

7e2fe07

Update changelog

d91a00e

PodsBot commented Nov 17, 2024

View reviewed changes

sc0ttkclark added 2 commits November 17, 2024 10:25

Pods 3.2.8

011a845

Update changelog

6b404d4

PodsBot commented Nov 17, 2024

View reviewed changes

Fix template name issue

a780c71

sc0ttkclark merged commit a780c71 into main Nov 17, 2024
11 of 18 checks passed

sc0ttkclark deleted the release/3.2.8 branch November 17, 2024 17:33

sc0ttkclark added this to the Pods 3.2.8 milestone Nov 17, 2024


		$params->custom_capabilities = (array) pods_var_raw( 'custom_capabilities', $params, array() );
		$params->custom_capabilities = (array) pods_v( 'custom_capabilities', $params, array() );

		if ( isset( $content ) ) {
		$has_php = false !== strpos( $content, '<?' );


		$has_objects_to_migrate = ! empty( $pod_templates ) \|\| ! empty( $pod_pages );

		// ajax_migrate

Pods 3.2.8 #7347

Pods 3.2.8 #7347

Conversation

PodsBot commented Aug 31, 2024

what-the-diff bot commented Aug 31, 2024 • edited Loading

PR Summary

PodsBot left a comment

Choose a reason for hiding this comment

PodsBot left a comment

Choose a reason for hiding this comment

PodsBot left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

PodsBot left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

PodsBot left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

PodsBot left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

PodsBot left a comment

Choose a reason for hiding this comment

PodsBot left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

PodsBot left a comment

Choose a reason for hiding this comment

PodsBot left a comment

what-the-diff bot commented Aug 31, 2024 •

edited

Loading